Skip to content

Update MegaMenuLayout.tsx#142

Closed
tiagov8 wants to merge 1 commit into
mainfrom
add-gap-between-menu-items
Closed

Update MegaMenuLayout.tsx#142
tiagov8 wants to merge 1 commit into
mainfrom
add-gap-between-menu-items

Conversation

@tiagov8

@tiagov8 tiagov8 commented Mar 26, 2026

Copy link
Copy Markdown
Member

From

Screenshot 2026-03-26 at 3 51 16 PM

To

Screenshot 2026-03-26 at 3 51 26 PM

@tiagov8 tiagov8 requested a review from Calinator444 March 26, 2026 22:53
@tiagov8 tiagov8 enabled auto-merge March 26, 2026 22:53
@Calinator444

Calinator444 commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

@tiagov8

Please remember to update the package.json file when making changes to the megamenu. as per the instructions.

@Calinator444

Calinator444 commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

@tiagov8 The megamenu already has this fix has already been applied to the mega-menu, but it's not visible on SSW.People because it's using an old version of this package.

CleanShot 2026-03-27 at 10 09 17

please review SSWConsulting/SSW.People#809

@tiagov8

tiagov8 commented Mar 26, 2026

Copy link
Copy Markdown
Member Author

Closing as per @Calinator444

we don't need this PR because SSW.People is using an old version of the megamenu.
The new version of the megamenu already has the fix
Here's a PR for updating SSW.People to the latest megamenu version
SSWConsulting/SSW.People#809

@tiagov8 tiagov8 closed this Mar 26, 2026
auto-merge was automatically disabled March 26, 2026 23:35

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants